Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding an option for the bacula file daemon to support custom message types #219

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

mi-reichel
Copy link
Contributor

Pull Request (PR) description

This adds a message parameter for the client to make customization of the logging configuration possible.

This Pull Request (PR) fixes the following issues

n/a

@mi-reichel mi-reichel force-pushed the mir_bacula-fd_messages_contrib branch from c67f0fc to 8e6dd13 Compare October 2, 2024 11:37
@mi-reichel
Copy link
Contributor Author

Hello,

just wanted to ask for the status.
Is there anything additional needed to have this merged?
If so please let me know so I can provide those details or change the pull request so it fits the needs.

Greetings
Michael

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like I started to do a review but left it pending. I can't push the minor style fix to your repo, can you integrate it? Other than that, LGTM!

README.md Outdated
Comment on lines 185 to 196
```puppet
class { 'bacula::client':
messages => { 'Standard-fd' => {
daemon => 'fd',
mname => 'Standard',
director => "${director}-dir = all, !skipped, !restored",
append => '"/var/log/bacula/bacula-fd.log" = all, !info, !skipped',
},
},
}
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow the Puppet Style guide:

Suggested change
```puppet
class { 'bacula::client':
messages => { 'Standard-fd' => {
daemon => 'fd',
mname => 'Standard',
director => "${director}-dir = all, !skipped, !restored",
append => '"/var/log/bacula/bacula-fd.log" = all, !info, !skipped',
},
},
}
```
```puppet
class { 'bacula::client':
messages => {
'Standard-fd' => {
daemon => 'fd',
mname => 'Standard',
director => "${director}-dir = all, !skipped, !restored",
append => '"/var/log/bacula/bacula-fd.log" = all, !info, !skipped',
},
},
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure we should keep this file, it is not maintainer by modulesync.

@mi-reichel mi-reichel force-pushed the mir_bacula-fd_messages_contrib branch from 8e6dd13 to 5467ee2 Compare January 6, 2025 10:02
@mi-reichel
Copy link
Contributor Author

@smortex
I have integrated the minor style fix.

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@smortex smortex merged commit 662d553 into voxpupuli:master Jan 7, 2025
5 checks passed
@smortex smortex added the enhancement New feature or request label Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants